Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FE] refactor: 리뷰 상세페이지에서 변경된 API를 연동 #297

Merged
merged 14 commits into from
Aug 12, 2024

Conversation

soosoo22
Copy link
Contributor

@soosoo22 soosoo22 commented Aug 11, 2024


🚀 어떤 기능을 구현했나요 ?

  • 리뷰 상세 페이지에서 변경된 API를 연동했습니다. 그리고 전반적인 디자인을 수정했습니다.

🔥 어떻게 해결했나요 ?

변경된 API에 맞게 인터페이스를 변경했습니다.

export interface Options {
  optionId: number;
  content: string;
  isChecked: boolean;
}

export interface OptionGroup {
  optionGroupId: number;
  minCount: number;
  maxCount: number;
  options: Options[];
}

export type QuestionType = 'CHECKBOX' | 'TEXT';

export interface QuestionData {
  questionId: number;
  required: boolean;
  questionType: QuestionType;
  content: string;
  optionGroup: OptionGroup | null;
  hasGuideline?: boolean;
  guideline?: string | null;
  answer?: string;
}

export interface DetailReviewSection {
  sectionId: number;
  header: string;
  questions: QuestionData[];
}

export interface DetailReviewData {
  formId: number;
  revieweeName: string;
  projectName: string;
  createdAt: string;
  sections: DetailReviewSection[];
}

리뷰 작성, 목록과 비슷하게 디자인을 변경해봤는데 다들 어떤 것 같나요🙂
스크린샷 2024-08-11 오전 10 46 11

Q. 개인적으로는 리뷰 상세 페이지에서는 가이드라인을 보여주지 않아도 될 것 같은데 다들 어떻게 생각하시나요?

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

리뷰미 화이팅!!!

@soosoo22 soosoo22 self-assigned this Aug 11, 2024
@soosoo22 soosoo22 linked an issue Aug 11, 2024 that may be closed by this pull request
1 task
@donghoony
Copy link
Contributor

가이드라인은 상세에서 필요없을지도 모르겠네요 ㅎㅎ

Copy link
Contributor

@BadaHertz52 BadaHertz52 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생했어!!!

Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생 많았어요~~

Copy link
Contributor

@chysis chysis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

바뀐 API에 맞게 적용되는 것 확인했어요. 몇 가지 간단한 수정 제안 사항을 코멘트로 남겼어요 😊

@@ -5,26 +5,101 @@ export const DETAILED_PAGE_MOCK_API_SETTING_VALUES = {
memberId: 2,
};

const ANSWER =
'림순의 바람은 그윽한 산들바람처럼 잔잔하게 흘러갔습니다. \n 눈부신 햇살이 그의 어깨를 감싸며, 푸른 하늘 아래 펼쳐진 들판을 바라보았습니다.\n 그의 마음은 자연의 아름다움 속에서 평온을 찾았고, 그 순간마다 삶의 소중함을 느꼈습니다.\n 그는 늘 그러한 순간들을 기억하며, 미래의 나날들을 기대했습니다. \n 바람은 여전히 그를 감싸며, 그의 마음 속 깊은 곳에 있는 꿈과 희망을 불러일으켰습니다.\n 림순은 미소 지으며 앞으로 나아갔습니다.림순의 바람은 그윽한 산들바람처럼 잔잔하게 흘러갔습니다. \n 눈부신 햇살이 그의 어깨를 감싸며, 푸른 하늘 아래 펼쳐진 들판을 바라보았습니다.\n 그의 마음은 자연의 아름다움 속에서 평온을 찾았고, 그 순간마다 삶의 소중함을 느꼈습니다.\n 그는 늘 그러한 순간들을 기억하며, 미래의 나날들을 기대했습니다. 림순의 바람은 그윽한 산들바람처럼 잔잔하게 흘러갔습니다. \n 눈부신 햇살이 그의 어깨를 감싸며, 푸른 하늘 아래 펼쳐진 들판을 바라보았습니다.\n 그의 마음은 자연의 아름다움 속에서 평온을 찾았고, 그 순간마다 삶의 소중함을 느꼈습니다.\n 그는 늘 그러한 순간들을 기억하며, 미래의 나날들을 기대했습니다. \n 바람은 여전히 그를 감싸며, 그의 마음 속 깊은 곳에 있는 꿈과 희망을 불러일으켰습니다.\n 림순은 미소 지으며 앞으로 나아갔습니다.림순의 바람은 그윽한 산들바람처럼 잔잔하게 흘러갔습니다. \n 눈부신 햇살이 그의 어깨를 감싸며, 푸른 하늘 아래 펼쳐진 들판을 바라보았습니다.\n 그의 마음은 자연의 아름다움 속에서 평온을 찾았고, 그 순간마다 삶의 소중함을 느꼈습니다.\n 그는 늘 그러한 순간들을 기억하며, 미래의 나날들을 기대했습니다. ';
const revieweeName = 'badahertz52';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상수는 UPPER_CASE로 작성해주세요!

questionId: 1,
required: true,
questionType: 'CHECKBOX',
content: `프로젝트 기간 동안, ${revieweeName}의 강점이 드러났던 순간을 선택해주세요. (1~2개)`,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

질문 내용 뒤에 (1~2개) 부분은 추후 제거하거나 하드코딩으로 명시해줘도 좋을 것 같아요.

Comment on lines +34 to +36
{/* {detailedReview.contents.map(({ id, question, answer }, index) => (
<ReviewSection key={id} question={question} answer={answer} index={index} />
))}
<KeywordSection keywords={detailedReview.keywords} index={detailedReview.contents.length} />
))} */}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

사용하지 않는 부분이라면 코드를 제거해도 좋겠네요.

@soosoo22 soosoo22 merged commit c58c6b8 into develop Aug 12, 2024
5 checks passed
@donghoony donghoony deleted the fe/refactor/275-detailed-review-page-update-api branch August 20, 2024 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] 리뷰 상세 페이지에서 변경된 API를 연동한다.
5 participants